Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Survey UI updates #17658

Merged
merged 9 commits into from
Oct 2, 2023
Merged

feat: Survey UI updates #17658

merged 9 commits into from
Oct 2, 2023

Conversation

smallbrownbike
Copy link
Contributor

Changes

  • Updates survey previews to match new survey designs
  • Adjusts default colors
  • Adds new survey options:
    • ratingButtonActiveColor (changes selected number & emoji rating background/fill color - only available on rating surveys)
    • borderColor
    • placeholder (only available on open surveys)
    • position (changes where the survey appears on the user's site - left, center, right)
  • Removes timeout on thank you message preview
    • Clicking "Close" now switches back to the main survey preview
  • Automatically adjusts preview text colors (black or white) based on the user's color options
Screenshot 2023-09-27 at 1 46 08 PM

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

5 snapshot changes in total. 0 added, 5 modified, 0 deleted:

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@liyiy liyiy requested review from liyiy and neilkakkar September 28, 2023 15:31
@@ -2097,15 +2097,16 @@ export enum SurveyType {
export interface SurveyAppearance {
backgroundColor?: string
submitButtonColor?: string
textColor?: string
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the other PR in the site app still supports textColor and descriptionTextColor, we should remove it there too then

Copy link
Contributor

@liyiy liyiy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM otherwise

@liyiy
Copy link
Contributor

liyiy commented Sep 28, 2023

There's some valid type errors in the code quality check, with Surveys.stories.tsx

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@liyiy liyiy merged commit fae1c4d into master Oct 2, 2023
72 checks passed
@liyiy liyiy deleted the survey-updates branch October 2, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants